Open
Conversation
BaeJinho4028
left a comment
BaeJinho4028
left a comment
There was a problem hiding this comment.
커밋과 코드가 좋다고 말씀드리긴 어려울 것 같습니다. 코드도 PR을 제출하려고 급하게 작성하신 모습도 보입니다. 차라리 늦더라도 본인 학습을 위해 더 퀄리티있게 작성하는게 낫지 않았을까요? 다른분들의 코드를 보며 좋은 기술들을 많이 훔쳐서 본인 기술로 만드시는 것을 추천 드립니다.
이렇게 과제를 마감에 급하게 간단하게 하게되면 얻어가는것도 적고 끝까지 유지하기도 어렵습니다. 저는 중고등학교부터 코딩 공부를 미리했으면 좋았을텐데라는 후회를 합니다. 현재 취업까지 많이 남으셨으니 위기감이 부족할 수 있지만, 나중에는 더욱 험난한 채용시장에서 취업하려면 조금이라도 일찍 준비해야합니다. 벌써 자바부터 공부하는 것이 어려운 과정인 것은 이해하지만, 좀더 공부에 몰두하셔서 저와 같은 후회를 안남기시기를 하는 바람에 말씀드렸습니다. 이번주차 과제 고생하셨습니다.
docs/README.md
Outdated
Comment on lines
1
to
4
| feat : 자동차 이름 입력 | ||
| feat : 이름 글자 5글자 이상시 오류 발생 | ||
| feat : 시도할 횟수 입력 | ||
| feat : 0~9중에서 랜덤한 정수 생성 No newline at end of file |
Comment on lines
+37
to
+44
| public class Random{ | ||
|
|
||
| } | ||
| @Override | ||
| public void runMain() { | ||
|
|
||
| Application.main(new String[]{}); | ||
|
|
Comment on lines
+7
to
+26
| class RandomNum{ | ||
| private int num = 0; | ||
| private int car_num = 0; | ||
| public int random_num(){ | ||
| num = Randoms.pickNumberInRange(0, 9); | ||
| if(num >= 4) car_num += 1; | ||
| return car_num; | ||
| } | ||
| } | ||
| class Carprint{ | ||
| StringBuilder car_position = new StringBuilder(); | ||
|
|
||
| public String print_car(int go_car_int){ | ||
| String car_position = "-".repeat(go_car_int); | ||
| return car_position; | ||
| } | ||
| } | ||
| class Winner { | ||
| //우승자 출력 | ||
| } |
There was a problem hiding this comment.
클래스들을 하나의 파일에 모두 작성하는 것은 좋지 않습니다.
Comment on lines
+53
to
+61
| for(int k = 0; k < input_num; k++){ | ||
| for(int j = 0; j < name.length; j++){ | ||
| go_number[j] = cars[j].random_num(); | ||
| } | ||
|
|
||
| for(int j = 0; j < name.length; j++){ | ||
| System.out.println(name[j] + " : " + p_cars[j].print_car(go_number[j])); | ||
| } | ||
| } |
There was a problem hiding this comment.
리뷰하는 3자의 입장에서 이러한 코드는 의미를 파악하기 어렵습니다. 잘 모듈화해보시고, 객체지향적인 설계를 해보시길 바랍니다.
Comment on lines
+1
to
+4
| feat : 자동차 이름 입력 | ||
| feat : 이름 글자 5글자 이상시 오류 발생 | ||
| feat : 시도할 횟수 입력 | ||
| feat : 0~9중에서 랜덤한 정수 생성하는 클래스 생성 No newline at end of file |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR을 제출하라는것을 몰라서 늦게 제출하게 되었습니다. 죄송합니다. java를 사용하는게 많이 미숙해서 정신없이 코드를 짠거 같아요. 2주차 강의를 토대로 많이 뜯어고쳐 보겠습니다.